Model custom path #1
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Signed-off-by: rsun19 <[email protected]>
* Nest SDK readthedocs documentation Signed-off-by: Anish Asthana <[email protected]> * Merge sdk OWNERS with KFP website approvers Signed-off-by: Anish Asthana <[email protected]> --------- Signed-off-by: Anish Asthana <[email protected]>
* chore: Adding Devin's DeepWiki to README Signed-off-by: Francisco Javier Arceo <[email protected]> using Devin's badge instead Signed-off-by: Francisco Javier Arceo <[email protected]> Adding badge and section: Signed-off-by: Francisco Javier Arceo <[email protected]> * adding warning and moved below documentation Signed-off-by: Francisco Javier Arceo <[email protected]> --------- Signed-off-by: Francisco Javier Arceo <[email protected]>
) Signed-off-by: rsun19 <[email protected]>
Signed-off-by: Michael <[email protected]>
…anges (kubeflow#11934) Signed-off-by: VaniHaripriya <[email protected]>
… versions (kubeflow#11952) * Add the name field for pipelines and pipeline versions The API only accepted display_name, which accepted arbitrary characters but this does not work well in Kubernetes native API mode (Kubernetes pipeline store) since object names must be a valid DNS subdomain. This commit adds the name field which is required, but for backwards compatibility, the API will accept just name or display_name and duplicate the set value to the unset field. For existing users, they will only see that there is an extra field of "name" on pipelines and pipeline versions that is the same value as "display_name". The database migration logic adds a new DisplayName column and sets the values to what was in the Name column. Confusingly, the API field "display_name" used to map the a database column of "Name". The UI now handles both name and display name but for a simpler user experience, the display name form fields when uploading a pipeline are hidden when not in Kubernetes native API mode (Kubernetes pipeline store) since there is no need to have a display name since there is no naming restrictions on the name. Signed-off-by: mprahl <[email protected]> * Add a better API error when a Kubernetes object name is invalid Signed-off-by: mprahl <[email protected]> --------- Signed-off-by: mprahl <[email protected]>
Signed-off-by: Laurent Goderre <[email protected]>
…amples/ffdl-seldon/source/seldon-pytorch-serving-image (kubeflow#11959) Bumps [torch](https://github.com/pytorch/pytorch) from 2.6.0 to 2.7.1. - [Release notes](https://github.com/pytorch/pytorch/releases) - [Changelog](https://github.com/pytorch/pytorch/blob/main/RELEASE.md) - [Commits](pytorch/pytorch@v2.6.0...v2.7.1) --- updated-dependencies: - dependency-name: torch dependency-version: 2.7.1 dependency-type: direct:production ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…beflow#11920) * fix(cache): Optimize cache expiration query, add index to db column, rm duplicate cache row insertions Signed-off-by: cbartram <[email protected]> * fix(cache): Use composite index for cache started_at_in_sec and id fields Signed-off-by: cbartram <[email protected]> Signed-off-by: cbartram <[email protected]> * chore(format): Fix formatting, rm commented code Signed-off-by: cbartram <[email protected]> Signed-off-by: cbartram <[email protected]> --------- Signed-off-by: cbartram <[email protected]>
) Add support for exit handlers to take input PipelineTaskFinalStatus. Signed-off-by: agoins <[email protected]>
…kubeflow#11960) Signed-off-by: Laurent Goderre <[email protected]>
Signed-off-by: John Thompson <[email protected]>
…ib/arena/docker (kubeflow#11963) Bumps [requests](https://github.com/psf/requests) from 2.18.4 to 2.32.4. - [Release notes](https://github.com/psf/requests/releases) - [Changelog](https://github.com/psf/requests/blob/main/HISTORY.md) - [Commits](psf/requests@v2.18.4...v2.32.4) --- updated-dependencies: - dependency-name: requests dependency-version: 2.32.4 dependency-type: direct:production ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
After making name and display_name separate fields in the API, the pipeline sample upload code needed to accomodate that, otherwise it would try to upload the pipeline without a display name, which is a required field. The backwards compatibility was handled at the API server level but this code bypassed the API server endpoints and used the resource manager directly. Signed-off-by: mprahl <[email protected]>
…nal-test (kubeflow#11966) Bumps [requests](https://github.com/psf/requests) from 2.32.2 to 2.32.4. - [Release notes](https://github.com/psf/requests/releases) - [Changelog](https://github.com/psf/requests/blob/main/HISTORY.md) - [Commits](psf/requests@v2.32.2...v2.32.4) --- updated-dependencies: - dependency-name: requests dependency-version: 2.32.4 dependency-type: indirect ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Signed-off-by: mprahl <[email protected]>
…ubeflow#11971) Bumps and [brace-expansion](https://github.com/juliangruber/brace-expansion). These dependencies needed to be updated together. Updates `brace-expansion` from 2.0.1 to 2.0.2 - [Release notes](https://github.com/juliangruber/brace-expansion/releases) - [Commits](juliangruber/brace-expansion@v2.0.1...v2.0.2) Updates `brace-expansion` from 1.1.11 to 2.0.2 - [Release notes](https://github.com/juliangruber/brace-expansion/releases) - [Commits](juliangruber/brace-expansion@v2.0.1...v2.0.2) --- updated-dependencies: - dependency-name: brace-expansion dependency-version: 2.0.2 dependency-type: indirect - dependency-name: brace-expansion dependency-version: 2.0.2 dependency-type: indirect ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…ubeflow#11972) Bumps [brace-expansion](https://github.com/juliangruber/brace-expansion) from 1.1.11 to 1.1.12. - [Release notes](https://github.com/juliangruber/brace-expansion/releases) - [Commits](juliangruber/brace-expansion@1.1.11...v1.1.12) --- updated-dependencies: - dependency-name: brace-expansion dependency-version: 1.1.12 dependency-type: indirect ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…server (kubeflow#11978) Bumps [brace-expansion](https://github.com/juliangruber/brace-expansion) from 1.1.11 to 1.1.12. - [Release notes](https://github.com/juliangruber/brace-expansion/releases) - [Commits](juliangruber/brace-expansion@1.1.11...v1.1.12) --- updated-dependencies: - dependency-name: brace-expansion dependency-version: 1.1.12 dependency-type: indirect ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…nal-test (kubeflow#11985) Bumps [protobuf](https://github.com/protocolbuffers/protobuf) from 4.25.3 to 4.25.8. - [Release notes](https://github.com/protocolbuffers/protobuf/releases) - [Changelog](https://github.com/protocolbuffers/protobuf/blob/main/protobuf_release.bzl) - [Commits](protocolbuffers/protobuf@v4.25.3...v4.25.8) --- updated-dependencies: - dependency-name: protobuf dependency-version: 4.25.8 dependency-type: indirect ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Signed-off-by: Humair Khan <[email protected]>
…kubeflow#11986) Bumps [protobuf](https://github.com/protocolbuffers/protobuf) from 4.25.4 to 4.25.8. - [Release notes](https://github.com/protocolbuffers/protobuf/releases) - [Changelog](https://github.com/protocolbuffers/protobuf/blob/main/protobuf_release.bzl) - [Commits](protocolbuffers/protobuf@v4.25.4...v4.25.8) --- updated-dependencies: - dependency-name: protobuf dependency-version: 4.25.8 dependency-type: indirect ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…gemaker (kubeflow#11993) Bumps [urllib3](https://github.com/urllib3/urllib3) from 1.26.15 to 2.5.0. - [Release notes](https://github.com/urllib3/urllib3/releases) - [Changelog](https://github.com/urllib3/urllib3/blob/main/CHANGES.rst) - [Commits](urllib3/urllib3@1.26.15...2.5.0) --- updated-dependencies: - dependency-name: urllib3 dependency-version: 2.5.0 dependency-type: direct:production ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Signed-off-by: zazulam <[email protected]>
This PR fixes several minor typos in backend/src/apiserver/resource/resource_manager.go: Changes "gabarage-collected" to "garbage-collected" Changes "successfull" to "successful" Fixes "successfully workflows runs" and "failed workflows runs" to "successful workflow runs" and "failed workflow runs" Corrects "reties" to "retries" These changes improve code clarity and documentation quality. No functional code changes were made. Signed-off-by: milosjava <[email protected]>
…te test to cover pipelines with outputs (kubeflow#12001) Signed-off-by: Helber Belmiro <[email protected]>
Signed-off-by: Humair Khan <[email protected]>
Signed-off-by: Helber Belmiro <[email protected]>
…w#11999) Bumps [urllib3](https://github.com/urllib3/urllib3) from 2.4.0 to 2.5.0. - [Release notes](https://github.com/urllib3/urllib3/releases) - [Changelog](https://github.com/urllib3/urllib3/blob/main/CHANGES.rst) - [Commits](urllib3/urllib3@2.4.0...2.5.0) --- updated-dependencies: - dependency-name: urllib3 dependency-version: 2.5.0 dependency-type: direct:production ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Signed-off-by: VaniHaripriya <[email protected]>
The source distribution produced does not include all of the files needed to rebuild the wheel. This change adds a MANIFEST.in file to cause the requirements.txt and requirements-dev.txt files to be included so when the setup.py tries to read them they are present. Signed-off-by: Doug Hellmann <[email protected]>
…line resources (kubeflow#12202) * Add configuration to conditionally disable shared pipeline resources Signed-off-by: Krishan Bhasin <[email protected]> * Add configuration to conditionally disable shared pipeline resources Signed-off-by: Krishan Bhasin <[email protected]> --------- Signed-off-by: Krishan Bhasin <[email protected]>
… (kubeflow#12295) add back additionalinfo as optional make k8s error logs more generic to protect against xss remove verbose logging for minio get artifact handler sanitize rather than remove resource names + revert backend error logs fix typo + add invalid resource name tests add valid namespace check for tensorboard update pipeline upload server responses + log verbose error messages remove error wrap from backend error logs minor updates to satisfy pre commit checks update tensor board file to ensure namespace is a str update frontend code format + backend test cases update pipeline upload test cases to pass pre commit fix minor merge typo fix minor merge typo Signed-off-by: JerT33 <[email protected]>
…n Tests (kubeflow#12335) Signed-off-by: VaniHaripriya <[email protected]>
* chore: Release 2.14.3 (kubeflow#12198) Signed-off-by: mprahl <[email protected]> Signed-off-by: Humair Khan <[email protected]> * chore: release 2.14.4 (kubeflow#12299) * fix(sdk): Align the versions on kfp[kubernetes] install (kubeflow#12283) It was errantly set to install kfp-kubernetes below 2 but the versions were recently aligned. Signed-off-by: mprahl <[email protected]> (cherry picked from commit f07e276) Signed-off-by: Humair Khan <[email protected]> * chore: Release 2.14.4 Signed-off-by: Humair Khan <[email protected]> --------- Signed-off-by: Humair Khan <[email protected]> Co-authored-by: Matt Prahl <[email protected]> * chore: release 2.14.5 automate requirements updates Signed-off-by: Humair Khan <[email protected]> * chore(release): bump version to 2.14.5 on master branch Signed-off-by: Humair Khan <[email protected]> * cleanup release docs Signed-off-by: Humair Khan <[email protected]> * update sdk ver for loaded components Signed-off-by: Humair Khan <[email protected]> * re-compile workflow yamls to align with ver update Signed-off-by: Humair Khan <[email protected]> --------- Signed-off-by: mprahl <[email protected]> Signed-off-by: Humair Khan <[email protected]> Co-authored-by: Matt Prahl <[email protected]>
Signed-off-by: Humair Khan <[email protected]>
Signed-off-by: Humair Khan <[email protected]>
Signed-off-by: Humair Khan <[email protected]>
…kubeflow#12339) The source distribution produced does not include all of the files needed to rebuild the wheel. This change adds a MANIFEST.in file to cause the requirements.txt file to be included so when the setup.py tries to read it, it is present. Signed-off-by: Doug Hellmann <[email protected]>
Signed-off-by: kaikaila <[email protected]>
* remove pin on protobuf 6 ver Signed-off-by: Humair Khan <[email protected]> * use requirements.in for kfp-k8s and spec Signed-off-by: Humair Khan <[email protected]> --------- Signed-off-by: Humair Khan <[email protected]>
) Signed-off-by: VaniHaripriya <[email protected]>
* chore: release 2.14.6 (kubeflow#12355) * chore: add publish package workflow (kubeflow#12344) Signed-off-by: Humair Khan <[email protected]> (cherry picked from commit 08ff96b) * chore: regenerate files for ver upgrade (kubeflow#12341) Signed-off-by: Humair Khan <[email protected]> (cherry picked from commit f689e86) * chore: update versioning policy (kubeflow#12342) Signed-off-by: Humair Khan <[email protected]> (cherry picked from commit d0e85fa) * chore: remove pin on protobuf 6 ver (kubeflow#12345) * remove pin on protobuf 6 ver Signed-off-by: Humair Khan <[email protected]> * use requirements.in for kfp-k8s and spec Signed-off-by: Humair Khan <[email protected]> --------- Signed-off-by: Humair Khan <[email protected]> (cherry picked from commit 06e6a44) * test for kfp-server-api codegen in ci Signed-off-by: Humair Khan <[email protected]> (cherry picked from commit b23aac8) * chore: Release 2.14.6 Signed-off-by: Humair Khan <[email protected]> * build wheels from sdist Signed-off-by: Humair Khan <[email protected]> * revert click version change Pin click on 8.1.8. Due to compatibility issues, cli tests fail on click>8.1.8 in Python >= 3.13. Signed-off-by: Humair Khan <[email protected]> --------- Signed-off-by: Humair Khan <[email protected]> * revert click version change Pin click on 8.1.8. Due to compatibility issues, cli tests fail on click>8.1.8 in Python >= 3.13. Signed-off-by: Humair Khan <[email protected]> * update requirements Signed-off-by: Humair Khan <[email protected]> * update golden files due to sdk version change Signed-off-by: Humair Khan <[email protected]> --------- Signed-off-by: Humair Khan <[email protected]>
* Fix CI to pass the TLS client flags This also adds wait logic for the cert-manager secret to be provisioned. Signed-off-by: mprahl <[email protected]> * Stop passing in explicit MySQL configurations to MLMD in TLS mode This conflicts with the configuration options. Signed-off-by: mprahl <[email protected]> * Disable CI for unimplemented test scenarios Pod to Pod TLS manifests are not yet implemented for Kubernetes Native API mode Signed-off-by: mprahl <[email protected]> * Build metadata-writer in CI to avoid TLS connection errors Signed-off-by: mprahl <[email protected]> * Provide --ca_cert_path only if one is set Signed-off-by: mprahl <[email protected]> * Use a clearer path for the custom CA files Having it be under /etc/pki/tls could override the system CAs. Signed-off-by: mprahl <[email protected]> * Add missing mlPipelineServiceTLSEnabled flag on SWF TLS manifests Signed-off-by: mprahl <[email protected]> * Add TLS connection info to importer Signed-off-by: mprahl <[email protected]> --------- Signed-off-by: mprahl <[email protected]>
…beflow#12350 (kubeflow#12351) Signed-off-by: JerT33 <[email protected]>
Signed-off-by: Humair Khan <[email protected]>
* add ml-piplines-ui-aritfact deployment Signed-off-by: Harshvir Potpose <[email protected]> * rm unused env vars Signed-off-by: Harshvir Potpose <[email protected]> * fix lifcycle policy Signed-off-by: Harshvir Potpose <[email protected]> * turn off artifact proxy Signed-off-by: Harshvir Potpose <[email protected]> * single-line ternary assignment Signed-off-by: Harshvir Potpose <[email protected]> * use descriptive variable name Signed-off-by: Harshvir Potpose <[email protected]> * Update sync.py fix variable names Signed-off-by: Julius von Kohout <[email protected]> * test artifact-proxy in ci Signed-off-by: Harshvir Potpose <[email protected]> * fix Signed-off-by: Harshvir Potpose <[email protected]> * fix Signed-off-by: Harshvir Potpose <[email protected]> * cleanup Signed-off-by: Harshvir Potpose <[email protected]> * add artifact proxy test Signed-off-by: Harshvir Potpose <[email protected]> * fix Signed-off-by: Harshvir Potpose <[email protected]> * Apply suggestions from code review Signed-off-by: Julius von Kohout <[email protected]> * Update test-artifact-proxy.sh Signed-off-by: Julius von Kohout <[email protected]> * Update test-artifact-proxy.sh Signed-off-by: Julius von Kohout <[email protected]> * Update test-artifact-proxy.sh Signed-off-by: Julius von Kohout <[email protected]> * fix Signed-off-by: Harshvir Potpose <[email protected]> * fix Signed-off-by: Harshvir Potpose <[email protected]> * fix artifact-proxy test Signed-off-by: Harshvir Potpose <[email protected]> * make it more readable Signed-off-by: Harshvir Potpose <[email protected]> * increase timeout Signed-off-by: Harshvir Potpose <[email protected]> * add debug Signed-off-by: Harshvir Potpose <[email protected]> * disable istio sidecar injection Signed-off-by: Harshvir Potpose <[email protected]> * add patch reference Signed-off-by: Harshvir Potpose <[email protected]> * remove debug statements Signed-off-by: Harshvir Potpose <[email protected]> * create profile after deployment Signed-off-by: Harshvir Potpose <[email protected]> * fix deploy script logic Signed-off-by: Harshvir Potpose <[email protected]> * move patches to kustomization.yaml Signed-off-by: Harshvir Potpose <[email protected]> * add namespace to seaweedfs service urls Signed-off-by: Harshvir Potpose <[email protected]> * Update kustomization.yaml Signed-off-by: Julius von Kohout <[email protected]> * Update .github/resources/manifests/multiuser/artifact-proxy/kustomization.yaml Co-authored-by: Matt Prahl <[email protected]> Signed-off-by: Julius von Kohout <[email protected]> * Update manifests/kustomize/base/installs/multi-user/pipelines-profile-controller/sync.py Co-authored-by: Matt Prahl <[email protected]> Signed-off-by: Harshvir Potpose <[email protected]> * Update manifests/kustomize/base/installs/multi-user/pipelines-profile-controller/sync.py Co-authored-by: Matt Prahl <[email protected]> Signed-off-by: Harshvir Potpose <[email protected]> * Update manifests/kustomize/base/installs/multi-user/pipelines-profile-controller/sync.py Co-authored-by: Matt Prahl <[email protected]> Signed-off-by: Harshvir Potpose <[email protected]> --------- Signed-off-by: Harshvir Potpose <[email protected]> Signed-off-by: Julius von Kohout <[email protected]> Co-authored-by: Julius von Kohout <[email protected]> Co-authored-by: Matt Prahl <[email protected]>
kubeflow#12364) Signed-off-by: Nelesh Singla <[email protected]>
…2374) Add missing envvar to the deployment of the ml-pipeline-ui-artifact to set the right namespace Signed-off-by: Alvaro Alonso <[email protected]>
…files for supporting Arm64 platform (kubeflow#12313) Update backend/Dockerfile with "ARG TARGETARCH" to get docker's "--platform" parameters. it supports backend/Dockerfile to download the packages matched CPU architecture. Update backend/Dockerfile.driver and backend/Dockerfile.launcher by removing the parameter, GOARCH, Update Dockerfile.driver Update Dockerfile.launcher Signed-off-by: jtu-ampere <[email protected]>
* Fix(backend): RetrievePodName with POD_NAMES v1 Signed-off-by: Alvaro Alonso <[email protected]> * fix: linting errors fix: Stylistic issues ST1000 ST1020 fix: Omit embedded fields from selector expression fix: Use strings.ReplaceAll Signed-off-by: Alvaro Alonso <[email protected]> --------- Signed-off-by: Alvaro Alonso <[email protected]>
Signed-off-by: JerT33 <[email protected]>
…low#12314) Signed-off-by: Giulio Frasca <[email protected]>
Signed-off-by: Googler <[email protected]> PiperOrigin-RevId: 821935037
Signed-off-by: agoins <[email protected]>
Signed-off-by: agoins <[email protected]>
Signed-off-by: agoins <[email protected]>
Signed-off-by: agoins <[email protected]>
Signed-off-by: agoins <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description of your changes:
Checklist: